inspector: Drop the per-object CSS editor

The global editor is good enough for all uses.

range: remove unneeded gtk_widget_queue_draw()

range: rename function

The function queues an allocation now, not a draw.

range: remove duplicated code

This is already called by range_grab_add().

scale: trivial code cleanup

scale: deprecate value-spacing style property

Nothing uses it now.

scale: port scale values to gadgets

And add a default color like it was before.
This also fixes other issues with scale values interacting with scale
mark labels, which were buggy at least since 3.18.

Updated Hungarian translation

Minor documentation improvement

https://bugzilla.gnome.org/show_bug.cgi?id=583780

spin button: Limit the entry width to reasonable values

When opening the value editor for any GtkAdjustment properties
in the inspector, the popover stretches out for miles, since
it reserves enough space to draw MAXDOUBLE. This is not useful.
Limit the space we reserve to 8 digits.

gtk-demo: Use <accelerator>

Add an example of <accelerator> use in the builder example.

builder: Allow 'primary' as a modifier value

Support aliases for flags values in GtkBuilder, and use this
to support the special value 'primary' for GdkModifierType
values.

https://bugzilla.gnome.org/show_bug.cgi?id=699891

range: simplify calculation

range: move declarations to inner block

Where they're needed.

range: use a fixed offset for mark "snap" size

Instead of making it dependent on the slider size.

Revert "range: use border box for slider area"

Since it causes problems with event coordinates.

This reverts commit 0883ff5eedf73b1197f2a49fb7e55ce227917335.

Updated Swedish translation

Updated Slovenian translation

Updated Slovenian translation

Make the settings test fail

Small thinko: We only get to see the test artifacts in
continuous if the test fails. So make it fail, temporarily.

Adwaita: fix glitch in fine-tune scrollbar theming

The fine-tune mode should not change the overall dimensions of the
scrollbar.

Updated Russian translation

Updated Slovenian translation

Updated Slovenian translation

Updated Lithuanian translation

win32: use a struct for the event source

This removes the event_poll_fd global variable and the (ab)use of
get_default_display. It is also more consistent with other backends.

Also store display

win32: fix "unused variable" warning

Updated Brazilian Portuguese translation

testsuite: Remove a test that isn't valid with csd

With csd, the allocation of the window will be larger than the
default size, since it includes borders and decorations. Just
remove this check.

Add a test that dumps settings

This is not much of a test, tbh. But it will give us a convenient
dump of all GTK+ settings in continuous, which helps with tracking
down test failures that might depend on settings.

status icon: Prevent size dithering

We're seeing loops where the size of some status icons constantly
dithers between 24 and 25. Since I couldn't track down exactly
where the one extra pixel comes from, just stop reacting
to single-pixel size changes.

https://bugzilla.gnome.org/show_bug.cgi?id=758893

Minor cleanup

The GtkStatusIcon::size-changed signal expects a boolean return.

Update a11y test output

These tests can now assume that the csd header bar will be there.

Run accessibility tests always with CSD

This avoids differences in the output that are caused
by the presence or absence of the header bar.

Updated Polish translation

Update Latvian translation

inspector: Avoid pointless selections

The CSS selector and the class hierarchy are purely
informative, no need to select anything here.

inspector: More grid lines

I had forgotten the css property list.

inspector: Some ui refinements

Remove pointless expanders from lists, add grid lines and,
in some places, tree lines.

Update expected output of a11y tests

The window type attribute shows up here now.
The caret mode changes also affect some tests.

a11y: Expose window type hint as attribute

This was an old request, maybe still useful.

https://bugzilla.gnome.org/show_bug.cgi?id=468122

messagedialog: Respect the gtk-keynav-use-caret setting

We used to always make the labels in message dialogs selectable,
which is a bit problematic wrt. to keynav - the label can
unexpectedly 'turn blue', which irritates some people.

With the new gtk-keynav-use-caret setting, we can now only
make the labels selectable when it is required for accessibilty
reasons.

gdk: Add an X setting for gtk-keynav-use-caret

This will let us toggle the use-caret setting session-wide.

text view: Respect gtk-keynav-use-caret

When the gtk-keynav-use-caret setting is TRUE, make the cursor
visible regardless of the cursor-visible property.

Add a gtk-keynav-use-caret mode setting

For accessibility reasons, it is useful to have a global setting
that makes carets visible in all text.

https://bugzilla.gnome.org/show_bug.cgi?id=602526
https://bugzilla.gnome.org/show_bug.cgi?id=762799

range: avoid setting slider coordinates to negative numbers

This can happen if the theme sets a negative margin, but the coordinate
should never be negative.

range: factor out a function

We're going to modify this in the next commit.

gtk-demo: Fix up the inline-toolbar example

The inline toolbar in the style classes demo should use
the icons toolbar-style to ensure that the buttons are square.

tool button: Update style classes

When changing tool button contents according to the toolbar-style
property, we need to update the style classes to ensure that the
visual style matches.

https://bugzilla.gnome.org/show_bug.cgi?id=760560

range: fix warning for gadget slider

The slider is not HFW/WFH - just pass -1 to get rid of the warnings.

gtkwindow: default size with fixed size windows

Allow fixed size windows with a default size to grow or shrink as the
content requires, but not smaller than the given default size.

https://bugzilla.gnome.org/show_bug.cgi?id=762974

gtkapplication-dbus: use application id instead of prgname for SM

gnome-session takes an "app id" for client and inhibitor registration,
This app id is supposed to be a desktop file id (complete with the
.desktop extension), but gtk+ currently uses g_get_prgname ().

This commit changes gtkapplication to use the application id instead,
which is a much more natural fit.  gnome-session is going to be updated
to stop using the .desktop extension, too, so everything is consistent.

https://bugzilla.gnome.org/show_bug.cgi?id=763106

gtkplacessidebar: add new other locations with flags signal

We weren't using the open flags on the other locations signal, which
makes impossible for applications like nautilus to act in the same
way that for any other location where the user can choose between
opening in the current view, in a new window or in a new tab.

Add a new signal with an open flags parameter and deprecate
the other-location signal.

https://bugzilla.gnome.org/show_bug.cgi?id=754743

Revert "gtkplacessidebar: add open flags to other locations"

This reverts commit 63a204e1c5b6c since it broke the API.
We will add a new signal instead in an upcomming patch.

https://bugzilla.gnome.org/show_bug.cgi?id=754743

Adwaita: treeview column header scrollbar top border

add a top border in vertical scrollbars in treeviews.

wayland: Don't crash with offscreen windows

Use gdk_window_get_effective_toplevel when looking for
a suitable transient parent, to skip over offscreen windows
that we might encounter in the window tree. This fixes
a crash in glade.

https://bugzilla.gnome.org/show_bug.cgi?id=763110

Adwaita: shrink scroolbars

erroneusly pushed 15px scrollbars with the previous commit, back
to 13px now.

Adwaita: fix a typo

Adwaita: scrollbar style rewrite

works correctly with the new gadgettified GtkRange Cosimo nicelly
delivered, as a bonus steppers are styled for overlay indicator
usage as well.

Updated Russian translation

range: add positional style classes to fill/highlight

Requested by Lapo.

x11: Trap errors around an XRandr call

Apparently, we can get X errors here. Ignore them.
https://bugzilla.gnome.org/show_bug.cgi?id=762907

gtkwindow: windows with a fixed size can shrink

One important aspect of non-resizable windows that we need to preserve
is that they shrink when their content requires less size.

Previous changes to allow the default size to be applied to fixed size
windows would have prevented all fixed size windows from shrinking when
their content requires less size.

Allow shrinking for fixed-size windows unless a default size was
specified.

https://bugzilla.gnome.org/show_bug.cgi?id=762974

Updated Hungarian translation

Adwaita: shrink back scale trough

like it was before the GtkRange gadgettification.

wayland: Clean up key repeat timeout

When returning G_SOURCE_REMOVE from a callback, we need
to reset the source id as well.

gadget: Add more details to warning

All the other warnings in this file include the css node name
and the gadget owners type. Be consistent about this.

gadget: Warn if for_size is too small

It does not make sense to pass a for_size into the size allocation
machinery that is smaller than the min-size in that direction.
Warn if it happens, so we can track it down when it occurs.

Adwaita: really fix scale with marks

added a min-height/width compensation too when in fine-tune mode,
so scale with marks and no labels doesn't make the scale node grow
when in fine-tune.

gtk-launch: add hack to close up dbus-daemon race

If gtk-launch exits before the program it starts fully activates,
then the dbus-daemon may avoid doing the activating method call.

This commit works around the problem by pinging the activated
application, and waiting for a reply.

This is similar to the gvfs-open bug here:

    https://bugzilla.gnome.org/show_bug.cgi?id=746534

https://bugzilla.gnome.org/show_bug.cgi?id=763103

Adwaita: fix a thinko wrt fine-tune scales

added a scale node padding compensation for the trough growth in
fine-tune mode, so now the scale node doesn't grow anymore when
mark labels are present.

icon helper: Properly update on style change

A GtkCssChange of NULL means 'everything changed!'.

entry: Make progress node invisible when appropriate

We were failing to do that, leading to progress not disappearing
anymore after it was initially shown, in the gtk3-widget-factory
entry progress example.

testscale: change example to use upwards labels

There's no example of that in testscale.

Adwaita: fix thinko in fine-tune scale selector

Marks always get class top/bottom, regardless of the scale orientation.

Updated Kazakh translation

Updated Kazakh translation

Check if XRRGetOutputInfo returned a null pointer.

Fixes bug 763023: in certain circumstances, XRRGetOutputInfo will return
a null pointer.  This commit adds a check to detect and handle this
return value.

expander: Reinstate rtl allocation behavior

In rtl mode, we must not just reverse the children,
but also allocate from the right. Use the newly introduced
box gadget api to achieve this.

https://bugzilla.gnome.org/show_bug.cgi?id=762945

check button: Reinstate rtl allocation behavior

In rtl mode, we must not just reverse the children,
but also allocate from the right. Use the newly introduced
box gadget api to achieve this.

https://bugzilla.gnome.org/show_bug.cgi?id=762945

box gadget: Add a way to allocate in reverse

This is needed to preserve expected allocation behavior
in rtl mode.

Updated POTFILES.skip

reftests: add a test for animated box shadow updating clip

This was just fixed by 3e0694284785153944255a0501e84a76c491e4b4

reftests: rename util file frame-inhibitor

We'll use the label-text-shadow-changes-modify-clip utils
more generically, lets name the file after its function.

revealer-size: Fix animation setting

Turn animations off before testing, for all test functions containing
"no_animations".

Updated Greek translation

Remove unused variable

GDK W32: Fix redrawing during drag-move with no composition

revealer: Don't use TRANSITION_NONE for disabled animations

If animations are disabled, the only difference we need to make is that
we don't have to start the transition. Size requests should remain the
same.

https://bugzilla.gnome.org/show_bug.cgi?id=762996

Add GtkRevealer sizing tests

https://bugzilla.gnome.org/show_bug.cgi?id=762996

Adwaita: fix a thinko about toolbar items spacing with margins

gtkwindow: Fix regression with fixed size windows

Previous commit to address the default size introduced a regression
with fixed size windows if no default size was given, the resulting
window would end up much smaller than its actual content.

Updated Korean translation

Updated Korean translation

Adwaita: uniform paddings on image-buttons

fixes https://bugzilla.gnome.org/show_bug.cgi?id=762811

Adwaita: use margin for spacing widgets on toolbars

kind of a workaround, this approach is pretty unmaintainable, I
hope we'll get a CSS property for spacing directly on the parent
container (a toolbar in this particular case).
See https://bugzilla.gnome.org/show_bug.cgi?id=762397

gtkwindow: Use default size even if not resizable

If a window is not resizable (with gtk_window_set_resizable ()),
the size given with gtk_window_set_default_size() is ignored.

The solution to this would be to use gtk_widget_set_size_request() but
that's a GtkWidget API and therefore does not take into account the
client side decorations when in use with GtkWindow.

Refactor the code so that gtk_window_set_default_size() (which is a
GtkWindow API) gives the expected result on non-resizable windows as
well.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=762974

W32: Prefer CSD by default

Will make GTK+ more willing to use CSD for all normal windows without
being asked to. Lack of desktop composition will, of course, prevent
it from using CSD (in theory).

GTK_CSD=0 will force CSD to NOT to be used whenever
possible (i.e. in cases where CSD is not specifically requested
by a window, by design).

https://bugzilla.gnome.org/show_bug.cgi?id=759899

GDK W32: Set default cursor from our own theme right away

Otherwise WM-dependent default cursor is used, which does not
match our theme. Worse, later GDK will realize that we have
our own left_ptr cursor and will apply it after all, making
the discrepancy even more noticeable.

https://bugzilla.gnome.org/show_bug.cgi?id=762902

GDK W32: Force correct mouse cursor for custom resize/move

* Explicitly grab the device, setting appropriate cursor on it.
* Fix gdk_device_virtual_set_window_cursor() to just set the
  cursor, without trying to check that mouse is over the given
  window. Also prevent it from immediately resetting cursor.
* Alse take into account things that happen in other parts of
  GDK - don't look for replacement cursor, GDK already did that,
  and don't create a default arrow cursor instead of NULL,
  GDK-W32 already did that up the stack as well.
  Warn about inappropriate cursor == NULL argument instead.

https://bugzilla.gnome.org/show_bug.cgi?id=762711